Skip to content

Conversation

WhitWaldo
Copy link
Contributor

Added build and publish workflows on the build.yml script

@WhitWaldo WhitWaldo self-assigned this Sep 9, 2025
Copy link
Contributor

@joebowbeer joebowbeer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some suggested changes

run: npm install

- name: Build package (install dependencies, lint, prettier, build)
run: npm run build
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Strictly speaking npm run build isn't needed or used since ts-jest is running tests

But running lint here would be a good idea

uses: codecov/codecov-action@ad3126e916f78f00edff4ed0317cf185271ccc2d # v5.4.2

publish:
needs: build
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe name that job "test" since it doesn't build but it tests

WhitWaldo and others added 5 commits September 18, 2025 17:20
Co-authored-by: Joe Bowbeer <[email protected]>
Signed-off-by: Whit Waldo <[email protected]>
Co-authored-by: Joe Bowbeer <[email protected]>
Signed-off-by: Whit Waldo <[email protected]>
Co-authored-by: Joe Bowbeer <[email protected]>
Signed-off-by: Whit Waldo <[email protected]>
Co-authored-by: Joe Bowbeer <[email protected]>
Signed-off-by: Whit Waldo <[email protected]>
Co-authored-by: Joe Bowbeer <[email protected]>
Signed-off-by: Whit Waldo <[email protected]>
Copy link
Contributor

@joebowbeer joebowbeer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@joebowbeer
Copy link
Contributor

Fixes #2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants